Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: drop python 3.7 and adopt 3.12 #1859

Merged
merged 1 commit into from
Nov 15, 2023
Merged

feat: drop python 3.7 and adopt 3.12 #1859

merged 1 commit into from
Nov 15, 2023

Conversation

roeap
Copy link
Collaborator

@roeap roeap commented Nov 14, 2023

Description

This change is mainly motivated by the current build failures on main. Since 3.7 is EOL and we have been meaning to drop it for a while, I thought maybe its good to give that a try before diving deeper into the inners of openssl build failures.

Closes #1703

@github-actions github-actions bot added the binding/python Issues for the Python package label Nov 14, 2023
@roeap roeap requested a review from ion-elgreco as a code owner November 14, 2023 21:46
@roeap
Copy link
Collaborator Author

roeap commented Nov 14, 2023

@rtyler - i believe you may have to take that one, as it requires updating the required tests ...

In the end we were running the 3.7 tests in a manylinux container that we used to use also for builds. we have since then changed to the github action, so i felt using the approach we use in the other tests would be OK.

Just hoping that we don't run into the same issues again in the next release, as the action may as well use a similar manylinux image. But hopefully one, where all build dependencies etc for openssl are compatible with the versions we need in the build...

Comment on lines -48 to -52
# actions/checkout@v3 is a node action, which runs on a fairly new
# version of node. however, manylinux environment's glibc is too old for
# that version of the node. so we will have to use v1 instead, which is a
# docker based action.
- uses: actions/checkout@v1
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can leave this as a TODO, but I think the proper thing to do would be use the Maturin action to build in manylinux like we do for the release jobs.

@rtyler rtyler enabled auto-merge (rebase) November 15, 2023 16:19
@rtyler rtyler merged commit d47b490 into delta-io:main Nov 15, 2023
24 checks passed
@roeap roeap deleted the drop-py37 branch November 15, 2023 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/python Issues for the Python package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop support for Python 3.7, test 3.12
4 participants